Skip to content

Fix static linking.#2342

Merged
istio-testing merged 2 commits intoistio:masterfrom
PiotrSikora:envoy-static
Aug 7, 2019
Merged

Fix static linking.#2342
istio-testing merged 2 commits intoistio:masterfrom
PiotrSikora:envoy-static

Conversation

@PiotrSikora
Copy link
Copy Markdown
Contributor

Broken in #2312.

Signed-off-by: Piotr Sikora piotrsikora@google.com

@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Aug 7, 2019
@istio-testing istio-testing added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Aug 7, 2019

# Link (statically) against libstdc++.
build --action_env=BAZEL_LINKLIBS=-l%:libstdc++.a
build --action_env=BAZEL_LINKOPTS=-lm:-static-libgcc
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I synchronized the other options with upstream while at it, but those lines are the real fix.

@duderino
Copy link
Copy Markdown

duderino commented Aug 7, 2019

/lgtm
/approve

Broken in istio#2312.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@PiotrSikora
Copy link
Copy Markdown
Contributor Author

Rebased on top of master to make sure all tests pass.

@istio-testing istio-testing merged commit 9e5640a into istio:master Aug 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants